I've assigned @valentinewallace as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

No memory exists. The diff is identical to my prior review pass, which found no issues. The changes are straightforward MSRV CI pins and an is_some_and → map().unwrap_or() replacement (since is_some_and requires Rust 1.70+, above the 1.63 MSRV), plus removal of now-unused imports.

No issues found.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.54%. Comparing base (b6f3ad7) to head (5b72cd4).

@@           Coverage Diff           @@
##              0.1    #4737   +/-   ##
=======================================
  Coverage   84.54%   84.54%           
=======================================
  Files         137      137           
  Lines       77617    77617           
  Branches    77617    77617           
=======================================
  Hits        65625    65625           
+ Misses       9949     9944    -5     
- Partials     2043     2048    +5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

CI still failing

I believe this is correct now. CI is now dependent on itself (its failing to build the release 0.1.10 lightning crate). rustfmt/linting/fuzz/build-tx-sync are legit-broken, but those aren't required to fix bindings.

Should we go ahead and fix those checks? Otherwise, we're committing to those checks failing on 0.1, etc branches moving forward?